Skip to content

ENH: Add STL-style iterators and size() member function to itk::Matrix#4894

Merged
dzenanz merged 1 commit intoInsightSoftwareConsortium:masterfrom
N-Dekker:Add-STL-interface-to-Matrix
Oct 24, 2024
Merged

ENH: Add STL-style iterators and size() member function to itk::Matrix#4894
dzenanz merged 1 commit intoInsightSoftwareConsortium:masterfrom
N-Dekker:Add-STL-interface-to-Matrix

Conversation

@N-Dekker
Copy link
Copy Markdown
Contributor

Forwarded iterators and size() from its internal m_Matrix. This commit enables calling itk::MakeFilled<MatrixType>(fillValue) to create a filled matrix.

Forwarded iterators and `size()` from its internal `m_Matrix`. This commit
enables calling `itk::MakeFilled<MatrixType>(fillValue)` to create a filled
matrix.
@github-actions github-actions bot added type:Enhancement Improvement of existing methods or implementation type:Testing Ensure that the purpose of a class is met/the results on a wide set of test cases are correct area:Core Issues affecting the Core module labels Oct 23, 2024
@N-Dekker N-Dekker marked this pull request as ready for review October 23, 2024 13:34
@dzenanz
Copy link
Copy Markdown
Member

dzenanz commented Oct 23, 2024

What is the use case for filled matrix? Usually we want an identity matrix.

@N-Dekker
Copy link
Copy Markdown
Contributor Author

What is the use case for filled matrix? Usually we want an identity matrix.

Moreover, I think an STL-style interface to itk::Matrix is nice to have, anyway 😃

@dzenanz dzenanz merged commit 7271903 into InsightSoftwareConsortium:master Oct 24, 2024
@thewtex
Copy link
Copy Markdown
Member

thewtex commented Oct 24, 2024

@N-Dekker awesome!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:Core Issues affecting the Core module type:Enhancement Improvement of existing methods or implementation type:Testing Ensure that the purpose of a class is met/the results on a wide set of test cases are correct

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants